Skip to content

Conversation

@ysmoradi
Copy link
Member

@ysmoradi ysmoradi commented Dec 18, 2025

closes #11900

Summary by CodeRabbit

  • Chores
    • Updated continuous integration and deployment workflows to utilize Xcode 26.2 for iOS and macOS builds, ensuring the latest development toolchain is used and maintaining compatibility across platforms.
    • Standardized build artifact naming conventions in deployment workflows for improved consistency across the CI/CD pipeline.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 18, 2025

Walkthrough

CI/CD workflow configurations updated across multiple repository locations to standardize Xcode version from 26.1 to 26.2. Artifact naming in the admin sample workflow changed from AdminPanelWeb to AdminPanel.

Changes

Cohort / File(s) Change Summary
Xcode version updates
.github/workflows/admin-sample.cd.yml, .github/workflows/blazorui.demo.cd.yml, .github/workflows/todo-sample.cd.yml, src/Templates/Boilerplate/Bit.Boilerplate/.github/workflows/cd-template.yml
Updated xcode-version from 26.1 to 26.2 across iOS/macOS build jobs
Artifact naming
.github/workflows/admin-sample.cd.yml
Renamed artifact from AdminPanelWeb to AdminPanel in upload step

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Verify Xcode 26.2 compatibility with existing build steps
  • Confirm artifact name change (AdminPanelWeb → AdminPanel) doesn't break downstream dependencies or deployment processes

Poem

🐰 Xcode marches on, version wise,
From 26.1 to 26.2 skies,
AdminPanel renamed with care,
CI workflows updated, fresh as air!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The PR title accurately reflects the main change: updating Xcode version from 26.1 to 26.2 across multiple workflow files.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between 607a7c1 and 85609cb.

📒 Files selected for processing (4)
  • .github/workflows/admin-sample.cd.yml (2 hunks)
  • .github/workflows/blazorui.demo.cd.yml (1 hunks)
  • .github/workflows/todo-sample.cd.yml (1 hunks)
  • src/Templates/Boilerplate/Bit.Boilerplate/.github/workflows/cd-template.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (18)
  • GitHub Check: build blazor hybrid (iOS-macOS)
  • GitHub Check: build blazor wasm standalone (small)
  • GitHub Check: build api
  • GitHub Check: build blazor hybrid (windows)
  • GitHub Check: build blazor wasm standalone (Offline database)
  • GitHub Check: build blazor hybrid (android)
  • GitHub Check: build blazor web
  • GitHub Check: build blazor wasm standalone (AOT)
  • GitHub Check: build blazor hybrid (windows)
  • GitHub Check: build api + blazor web
  • GitHub Check: build blazor hybrid (iOS-macOS)
  • GitHub Check: build blazor hybrid (android)
  • GitHub Check: build blazor wasm standalone
  • GitHub Check: build blazor hybrid (windows)
  • GitHub Check: build blazor hybrid (iOS-macOS)
  • GitHub Check: build api + blazor web
  • GitHub Check: build blazor hybrid (android)
  • GitHub Check: build Bit.Templates
🔇 Additional comments (4)
src/Templates/Boilerplate/Bit.Boilerplate/.github/workflows/cd-template.yml (1)

222-222: LGTM!

Xcode version update is consistent with other workflow files.

.github/workflows/blazorui.demo.cd.yml (1)

167-167: LGTM!

Xcode version update is consistent with other workflow files.

.github/workflows/admin-sample.cd.yml (2)

296-296: LGTM!

Xcode version update is consistent with other workflow files.


87-87: No downstream dependencies reference the old artifact name.

Verification confirms the artifact name change from AdminPanelWeb to AdminPanel is safe. A comprehensive search of the repository found no references to the old artifact name, and the new AdminPanel name is properly configured and in use throughout the workflow.

@msynk msynk changed the title Use xcode 26.2 across projects (#19000) Use xcode 26.2 across projects (#11900) Dec 19, 2025
@ysmoradi ysmoradi merged commit 89d1cdd into bitfoundation:develop Dec 19, 2025
20 checks passed
@ysmoradi ysmoradi deleted the 19000 branch December 19, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Projects must switch to xcode 26.2

1 participant